Skip to content

Conversation

@prdoyle
Copy link
Contributor

@prdoyle prdoyle commented May 30, 2025

Add an initial TestPolicyManager with barebones functionality. Neither TestPolicyManager nor TestPathLookup actually do much of anything yet.

I've peeled this into its own PR because I was actually surprised how much I needed to put into TestPolicyManager.isTrustedSystemClass and I just wanted to check in with the team.

See ES-11597.

@prdoyle prdoyle requested a review from a team as a code owner May 30, 2025 20:53
@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.1.0 labels May 30, 2025
@prdoyle prdoyle added >test Issues or PRs that are addressing/adding tests auto-backport Automatically create backport pull requests when merged v8.19.0 :Core/Infra/Entitlements Entitlements infrastructure labels May 30, 2025
@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team and removed needs:triage Requires assignment of a team area label labels May 30, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Contributor

@jdconrad jdconrad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@prdoyle prdoyle merged commit c633345 into elastic:main Jun 2, 2025
18 checks passed
@prdoyle prdoyle deleted the test-policy-manager branch June 2, 2025 17:08
prdoyle added a commit to prdoyle/elasticsearch that referenced this pull request Jun 2, 2025
* Initial TestPolicyManager implementation

* The forbidden APIs check is not messing around
elasticsearchmachine pushed a commit that referenced this pull request Jun 2, 2025
* Initial TestPolicyManager implementation

* The forbidden APIs check is not messing around
mridula-s109 pushed a commit that referenced this pull request Jun 2, 2025
* Initial TestPolicyManager implementation

* The forbidden APIs check is not messing around
mridula-s109 pushed a commit to mridula-s109/elasticsearch that referenced this pull request Jun 3, 2025
* Initial TestPolicyManager implementation

* The forbidden APIs check is not messing around
joshua-adams-1 pushed a commit to joshua-adams-1/elasticsearch that referenced this pull request Jun 3, 2025
* Initial TestPolicyManager implementation

* The forbidden APIs check is not messing around
Samiul-TheSoccerFan pushed a commit to Samiul-TheSoccerFan/elasticsearch that referenced this pull request Jun 5, 2025
* Initial TestPolicyManager implementation

* The forbidden APIs check is not messing around
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :Core/Infra/Entitlements Entitlements infrastructure Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants